The null check is broken since it will throw a Nullpointer itself. The reason is that a method is called on the object when it is null. It is likely that you used || instead of && or vice versa.

<p>
This rule is deprecated, use {rule:squid:S1697} instead.
</p>
